Skip to content

Use AI to model just a single section#2591

Merged
robertbrignull merged 1 commit intomainfrom
robertbrignull/data-llm-section
Jul 7, 2023
Merged

Use AI to model just a single section#2591
robertbrignull merged 1 commit intomainfrom
robertbrignull/data-llm-section

Conversation

@robertbrignull
Copy link
Copy Markdown
Contributor

@robertbrignull robertbrignull commented Jul 7, 2023

Hooks up the "Model with AI" in each collapsible section to work on just methods from that model. Follows a very similar approach to the "save" button. This thankfully is quite easy since we pass all the data back to the extension host, so we can just send only the data for that model. From then on the extension host sends back some modelled methods and it all works the same as it did before.

The old "Generate using LLM" in the header still works on everything. This button will be removed at a later date once we've finished redesigning the collapsible sections.

Note: the "unsaved" label is a little broken and doesn't update correctly when modelling with AI. It's already broken before this change too so so my rough plan was to go forward with this PR and address it in a separate PR. That is, to make the "unsaved" status is affected correctly when we get new changes from "model from source" or "model with LLM". But just say if you'd prefer to delay this PR and wait for that fix first.

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

Copy link
Copy Markdown
Contributor

@charisk charisk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🎉

@robertbrignull robertbrignull enabled auto-merge July 7, 2023 15:11
@robertbrignull robertbrignull merged commit 2525ae8 into main Jul 7, 2023
@robertbrignull robertbrignull deleted the robertbrignull/data-llm-section branch July 7, 2023 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants